Skip to content

Conversation

@Fluf22
Copy link
Collaborator

@Fluf22 Fluf22 commented Feb 17, 2025

🧭 What and Why

🎟 JIRA Ticket: DI-3413

Changes included:

  • Swift
  • Scala
  • Kotlin
  • Dart
  • PHP
  • Javascript
  • Python
  • Java
  • C#
  • Go
  • Ruby

@Fluf22 Fluf22 requested a review from a team as a code owner February 17, 2025 11:22
@Fluf22 Fluf22 requested review from millotp and shortcuts February 17, 2025 11:22
@Fluf22 Fluf22 self-assigned this Feb 17, 2025
@algolia-bot
Copy link
Collaborator

algolia-bot commented Feb 17, 2025

✔️ Code generated!

Name Link
🪓 Triggered by b52e2fc781c2ccfedd0ebb4efa54e7a61a3cc3c1
🍃 Generated commit 42efce0e864ce250c560a0e1a41f803e1135eb22
🌲 Generated branch generated/feat/more-generated-guides
📊 Benchmark results

Benchmarks performed on the method using a mock server, the results might not reflect the real-world performance.

Language Req/s
javascript 1676
go 1628
php 1479
csharp 1405
java 1086
python 1078
ruby 837
swift 766

@Fluf22 Fluf22 force-pushed the feat/more-generated-guides branch from b338e4a to f16f483 Compare February 17, 2025 17:46
Copy link
Contributor

@millotp millotp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huuuuuuge ! thanks a lot for doing this !


self.wait_task_time_before_retry: Optional[int] = None
self.headers: Optional[Dict[str, str]] = None
self.headers: Dict[str, str] = {}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this change ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because ultimately we always use it as a dict, it's never treated like it can be None
So setting it to an empty dict from the beginning ease the DX

@Fluf22 Fluf22 merged commit 5dd667b into main Feb 19, 2025
28 checks passed
@Fluf22 Fluf22 deleted the feat/more-generated-guides branch February 19, 2025 15:09
algolia-bot added a commit that referenced this pull request Feb 19, 2025
algolia-bot added a commit to algolia/algoliasearch-client-python that referenced this pull request Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants